Skip to content

feat(roadmap): 완료 콜백을 result 포함 /roadmap/complete로 전송#88

Closed
Youcu wants to merge 1 commit into
mainfrom
feat/roadmap-webhook-result
Closed

feat(roadmap): 완료 콜백을 result 포함 /roadmap/complete로 전송#88
Youcu wants to merge 1 commit into
mainfrom
feat/roadmap-webhook-result

Conversation

@Youcu

@Youcu Youcu commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

배경

로드맵 완료 콜백이 포폴/자소서용 공용 notify_be(POST /api/v1/ai/jobs/complete)를 쓰고 있어:

  • PDF가 없는 로드맵은 status항상 ERROR로 전송됨(공용 로직: status="DONE" if output_pdf_url else "ERROR").
  • 평가 result가 BE로 전달되지 않음(공용 페이로드엔 result 없음).

→ BE가 로드맵 결과를 받아 FE로 흘릴 수 없었음(BE↔FE는 relay 정합 완료, 이 콜백만 비어 있던 상태).

변경

  • webhook.notify_be_roadmap(ai_job_id, status, result, error_message) 추가 → POST {BE}/api/v1/ai/roadmap/complete (헤더 X-INTERNAL-API-KEY 동일).
  • run_assess_task finally: 성공 시 status=DONE+result, 실패 시 ERROR+error_messagenotify_be_roadmap 호출(공용 notify_be 대신).
  • 공용 notify_be(PDF 파이프라인용)는 그대로 유지 — 변경 없음.

BE는 /roadmap/complete(completeRoadmapJob)에서 result를 result_json으로 저장 후 SSE/폴링으로 FE에 전달(BE/FE PR과 짝).

검증

  • python -m py_compile 통과(문법). 공용 notify_bepdf_pipeline에서 계속 사용 — 무영향.

비고

  • 베이스를 main으로 두었습니다. 팀 flow가 develop 우선이면 retarget 해주세요.
  • result는 merge=false면 평가 배열, merge=true면 단건 배열로 전달(기존 result 구조 그대로).

🤖 Generated with Claude Code

로드맵은 PDF가 없어 공용 notify_be(/api/v1/ai/jobs/complete)로는 status가 항상
ERROR가 되고 평가 result도 BE에 전달되지 않았다. BE가 결과를 받아 FE로 흘릴 수 있게
로드맵 전용 콜백을 추가한다:

- webhook.notify_be_roadmap(ai_job_id, status, result, error_message)
  → POST {BE}/api/v1/ai/roadmap/complete (X-INTERNAL-API-KEY 동일)
- run_assess_task finally: 성공 시 status=DONE+result, 실패 시 ERROR+error_message로
  notify_be_roadmap 호출(공용 notify_be 대신).

공용 notify_be(포폴/자소서 PDF 경로)는 그대로 유지. BE는 completeRoadmapJob에서
result를 result_json으로 저장 후 SSE/폴링으로 FE에 전달.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@Youcu

Youcu commented Jun 4, 2026

Copy link
Copy Markdown
Contributor Author

develop에 이미 notify_be_roadmap이 있고(다른 시그니처), 이 PR은 그것을 main 기준으로 중복 정의해 #87(develop→main) 머지 시 충돌합니다. 진짜 빠진 건 'run_assess_task → notify_be_roadmap 연결'이라, 그 한 곳만 고치는 #89(→develop)로 대체합니다. 닫습니다.

@Youcu Youcu closed this Jun 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant